Skip to content

GitHub Action for RAT check #173

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 10 commits into from
Aug 12, 2025
Merged

GitHub Action for RAT check #173

merged 10 commits into from
Aug 12, 2025

Conversation

DaanHoogland
Copy link
Contributor

No description provided.

@DaanHoogland
Copy link
Contributor Author

DaanHoogland commented Aug 6, 2025

I guess a make target that calls the rat is better…
I want to also add a/the definition of a list of excludes to the makefile

Copy link

github-actions bot commented Aug 8, 2025

✅ Build complete for PR #173.

🔗 Download the cmk binaries (expires on August 22, 2025)

@DaanHoogland
Copy link
Contributor Author

@weizhouapache @shwstppr , I finally got my head around the excludes syntax. can you guys have a look? If alright I will create similar PRs for other cloudstack side projects.

@shwstppr
Copy link
Contributor

@DaanHoogland do we have a negative test (like valid go file missing license) somewhere?
Otherwise looks good

@DaanHoogland DaanHoogland merged commit 02ab307 into main Aug 12, 2025
6 checks passed
@DaanHoogland DaanHoogland deleted the rat-checks branch August 12, 2025 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants